Skip to content

Symfony Route annotation needs equal sign, not colon#3241

Merged
TomasVotruba merged 5 commits intorectorphp:masterfrom
stephanvierkant:symfony-route-options
May 4, 2020
Merged

Symfony Route annotation needs equal sign, not colon#3241
TomasVotruba merged 5 commits intorectorphp:masterfrom
stephanvierkant:symfony-route-options

Conversation

@stephanvierkant
Copy link
Copy Markdown
Contributor

See #3225

@stephanvierkant
Copy link
Copy Markdown
Contributor Author

stephanvierkant commented Apr 29, 2020

@TomasVotruba I see you've refactored an important part, but this PR isn't merged yet. Are you planning to fix this issue, or can I help?

@TomasVotruba
Copy link
Copy Markdown
Member

TomasVotruba commented Apr 29, 2020

Help is needed, I have no idea how to fix it, and other issues are in my to-do list.

@TomasVotruba
Copy link
Copy Markdown
Member

If the CI gets green, I'd be happy to merge it :)

@stephanvierkant stephanvierkant changed the title Symfony route with expose option (FosJsRoutingBundle) Symfony Route annotation needs equal sign, not colon Apr 30, 2020
@TomasVotruba
Copy link
Copy Markdown
Member

I'll try to solve this locally. Thanks for the PR!

@TomasVotruba TomasVotruba merged commit 7b4203e into rectorphp:master May 4, 2020
TomasVotruba added a commit that referenced this pull request Dec 22, 2022
rectorphp/rector-src@3c168f0 [CodingStyle] Skip new line /\r\n|\r|\n/i on ConsistentPregDelimiterRector (#3241)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants